Skip to content

ENT-12129: Added full hosts delete and 2FA support to distributed cleanup script (3.24.x)#3116

Merged
craigcomstock merged 2 commits into
cfengine:3.24.xfrom
aleksandrychev:ENT-12129_3.24.x
May 13, 2026
Merged

ENT-12129: Added full hosts delete and 2FA support to distributed cleanup script (3.24.x)#3116
craigcomstock merged 2 commits into
cfengine:3.24.xfrom
aleksandrychev:ENT-12129_3.24.x

Conversation

@aleksandrychev

@aleksandrychev aleksandrychev commented Apr 3, 2026

Copy link
Copy Markdown
Contributor

No description provided.

Ticket: ENT-12129

ChangeLog: Changed distributed_cleanup.py to issue a real DELETE FROM __hosts instead of soft deletion via INSERT with a deleted timestamp
Signed-off-by: Ihor Aleksandrychiev <ihor.aleksandrychiev@northern.tech>
(cherry picked from commit 5950d3d)
@aleksandrychev

Copy link
Copy Markdown
Contributor Author

@cf-bottom jenkins please

@cf-bottom

Copy link
Copy Markdown

@aleksandrychev

Copy link
Copy Markdown
Contributor Author

@cf-bottom jenkins

@cf-bottom

Copy link
Copy Markdown

@olehermanse olehermanse requested a review from craigcomstock May 6, 2026 14:42
@craigcomstock

Copy link
Copy Markdown
Contributor

@cf-bottom jenkins

@craigcomstock craigcomstock left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the deployment test needed changing, I made a PR for that and mention it here so that jenkins should use it... if that passes we can merge this.

@cf-bottom

Copy link
Copy Markdown

@craigcomstock

Copy link
Copy Markdown
Contributor

@cf-bottom previous jenkins run flaked, try again please

@cf-bottom

Copy link
Copy Markdown

@craigcomstock

Copy link
Copy Markdown
Contributor

previous jenkins run was flaky, @cf-bottom jenkins please
github failure might be due to recent release, will retry.

@cf-bottom

Copy link
Copy Markdown

…anup setup

Ticket: ENT-12129

ChangeLog: Title
Signed-off-by: Ihor Aleksandrychiev <ihor.aleksandrychiev@northern.tech>
(cherry picked from commit 1e3db88)
@craigcomstock

Copy link
Copy Markdown
Contributor

@cf-bottom jenkins

@cf-bottom

Copy link
Copy Markdown

@craigcomstock

Copy link
Copy Markdown
Contributor

deployment tests failed due to including the system-testing PR reference which was already merged. I have rebuilt each of those three jobs without the PR reference.

@craigcomstock

Copy link
Copy Markdown
Contributor

All rebuilds of deployment test jobs passed so will merge.

@craigcomstock craigcomstock merged commit da2e63f into cfengine:3.24.x May 13, 2026
45 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants